Skip to content

Checked containers reject ctype element - #152

Closed
robertmorelli wants to merge 1 commit into
facebookincubator:mainfrom
robertmorelli:check-not-ctype
Closed

Checked containers reject ctype element#152
robertmorelli wants to merge 1 commit into
facebookincubator:mainfrom
robertmorelli:check-not-ctype

Conversation

@robertmorelli

@robertmorelli robertmorelli commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

fix for #151
I believe that any CheckedList[ctype] or CheckedDict[ctype] is not supported
This enforces that at compile time

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Aug 26, 2026

@DinoV DinoV left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@meta-codesync

meta-codesync Bot commented Sep 3, 2026

Copy link
Copy Markdown

@DinoV has imported this pull request. If you are a Meta employee, you can view this in D118677602.

@meta-codesync meta-codesync Bot closed this in 4dcb734 Sep 3, 2026
@meta-codesync meta-codesync Bot added the Merged label Sep 3, 2026
@meta-codesync

meta-codesync Bot commented Sep 3, 2026

Copy link
Copy Markdown

@DinoV merged this pull request in 4dcb734.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants